Release 0.3.7 with native hooks in public uploads - #15
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
PR Summary by QodoRelease 0.3.7 with lifecycle hooks in public uploads
AI Description
Diagram
High-Level Assessment
Files changed (8)
|
Code Review by Qodo
1. Homebrew users get unsupported hooks
|
| { | ||
| "name": "tree-ring-memory", | ||
| "version": "0.3.6", | ||
| "version": "0.3.7", |
There was a problem hiding this comment.
1. Users see the wrong release version 🐞 Bug ≡ Correctness
The manifest version is bumped to 0.3.7, while the release introduction in README.md still identifies the plugin as 0.3.6. Anyone using the README to verify the installed or submitted release receives metadata that conflicts with both packaged manifests and the submission dossier.
Agent Prompt
## Issue description
The release manifests and submission dossier declare version `0.3.7`, but the README still presents the plugin as version `0.3.6`.
## Issue Context
This release updates both packaged manifests to `0.3.7`; user-facing release documentation should identify the same version. Add validation if practical so future release bumps cannot leave this copy stale.
## Fix Focus Areas
- README.md[9-12]
- scripts/validate-plugin.sh[69-70]
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
| for name in ("codex-hooks.json", "codex-hook.sh"): | ||
| path = PLUGIN / "hooks" / name | ||
| write_file(archive, path, path.relative_to(PLUGIN)) |
There was a problem hiding this comment.
2. Homebrew users get unsupported hooks 🔗 Cross-repo conflict ≡ Correctness
build() now includes native lifecycle hooks that require Tree Ring Memory CLI 0.15.6 or newer, while the live Homebrew formula still installs 0.15.3. When a Homebrew user installs this public upload without a project-local binary, codex-hook.sh resolves the outdated executable from PATH, reaching unsupported automatic recall and checkpoint behavior.
Agent Prompt
## Issue description
The public upload now activates lifecycle hooks requiring CLI 0.15.6+, but the live Homebrew installation path supplies CLI 0.15.3. Prevent the hook from invoking an unsupported Homebrew runtime and coordinate a formula update to at least 0.15.6, preferably the pinned 0.15.7 release.
## Issue Context
The hook prefers a project-local executable but otherwise resolves `tree-ring` from `PATH`, where Homebrew installs version 0.15.3. Add an explicit compatibility check or graceful fallback until the Homebrew formula is updated.
## Fix Focus Areas
- hooks/codex-hook.sh[21-26]
- packaging/build-codex-skills-only.py[40-42]
- /cross_repos/homebrew-tree-ring/Formula/tree-ring.rb[4-18]
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
Public uploads now retain the same startup and stop hooks as the Git package, including executable script permissions. Add deterministic archive checks and update the submission dossier to the current OpenAI contract.
Validation: wrapper, lifecycle, and public archive checks pass; the pinned runtime remains 0.15.7. https://developers.openai.com/plugins/guides/submit-claude-plugin
High-level PR Summary
This release (version 0.3.7) updates the public upload package to now include native lifecycle hooks (SessionStart, SubagentStart, Stop, SubagentStop) that were previously only available in the Git repository version. The build script now preserves executable permissions for hook scripts, adds deterministic archive validation, and updates documentation to reflect that both distribution channels (Git and public ZIP upload) now provide the same native Codex lifecycle capabilities. The submission documentation has been updated to align with current OpenAI compatibility guidelines.
⏱️ Estimated Review Time: 5-15 minutes
💡 Review Order Suggestion
packaging/codex-skills-only/.codex-plugin/plugin.json.codex-plugin/plugin.jsonpackaging/build-codex-skills-only.pyscripts/validate-upload.pyscripts/validate-plugin.shSUBMISSION.mdREADME.mdskills/tree-ring-memory/SKILL.md